Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Tech] Fixe les problèmes de style après suppression de mini.css #2722

Merged
merged 12 commits into from
Dec 1, 2023

Conversation

ivangabriele
Copy link
Member

Linked issues

None


  • Tests E2E (Cypress)

@ivangabriele ivangabriele changed the title [Tech] Fixe les problèmes de style après suppression de min.css [Tech] Fixe les problèmes de style après suppression de mini.css Nov 28, 2023
@ivangabriele ivangabriele force-pushed the ivan/fix-style-issues-following-min-css-removal branch from 80a27b3 to adfbf33 Compare November 28, 2023 05:37
Copy link
Collaborator

@louptheron louptheron left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah oui beaucoup de passage en Ts !
Je teste en local et je valide

@louptheron
Copy link
Collaborator

louptheron commented Nov 28, 2023

Alors il y a encore quelques styles de cassés, voici les screenshots :

Dans le backoffice REG (marges et champs inputs trop grrands):
Screenshot from 2023-11-28 10-32-39
Screenshot from 2023-11-28 10-32-13
Nom de texte réglementtaire 1 non-visible (visible si on la passe la souris dessus)
Screenshot from 2023-11-28 10-32-02
Screenshot from 2023-11-28 10-31-45
chevauchement du texte :
Screenshot from 2023-11-28 10-31-33
décalage avec une bande blanche en bas de la page quand on ouvre des couche sréglementaires :
Screenshot from 2023-11-28 10-29-22
souci de marges :
Screenshot from 2023-11-28 10-27-42
Screenshot from 2023-11-28 10-27-21
également un souci de margin top
Screenshot from 2023-11-28 10-25-31

Dispo pour voir en détail les diff si besoin!

@ivangabriele ivangabriele force-pushed the ivan/fix-style-issues-following-min-css-removal branch from 195d2ea to 12677a2 Compare November 30, 2023 13:07
@@ -73,12 +73,6 @@ function UnmemoizedRegulatoryTopic({
const onMouseEnter = () => !isOver && setIsOver(true)
const onMouseLeave = () => isOver && setIsOver(false)

useLayoutEffect(() => {
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Warning

Attention à ce changement.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

On n'a plus cette fonctonnalité de scroll ? C'était ce qui causait le bandeau blanc ?

Copy link
Member Author

@ivangabriele ivangabriele Nov 30, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

C'est en effet ça qui produit un blanc. J'ai expérimenté de le mettre dans un setTimeout que j'ai même monté jusqu'à 250 et ça ne changeait rien. Je pense que ça a un rapport avec le temps d'animation d'ouverture de la liste des couches mais je n'en suis pas sûr. Sur Env aussi c'était un soucis de scoll mais le fix est différent (et Env n'a pas d'animation d'ouverture je crois) : MTES-MCT/monitorenv#993.

@ivangabriele ivangabriele force-pushed the ivan/fix-style-issues-following-min-css-removal branch from ac6cf33 to 3e4c2c3 Compare November 30, 2023 16:37
Copy link

sonarcloud bot commented Dec 1, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 40 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@louptheron louptheron merged commit bc1c7f9 into master Dec 1, 2023
25 checks passed
@louptheron louptheron deleted the ivan/fix-style-issues-following-min-css-removal branch December 1, 2023 11:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants